Skip to content

Comments

chore: use custom GitHub App for update PRs so tests can run#25

Merged
niklashigi merged 2 commits intomainfrom
change-workflow
Feb 20, 2026
Merged

chore: use custom GitHub App for update PRs so tests can run#25
niklashigi merged 2 commits intomainfrom
change-workflow

Conversation

@maakle
Copy link
Collaborator

@maakle maakle commented Feb 19, 2026

Note

Medium Risk
Changes CI authentication/permissions and workflow execution path; misconfiguration of app secrets or token scopes could break SDK generation or PR creation.

Overview
Updates the SDK Generation GitHub Actions workflow to run SDK generation directly in the job (rather than calling the shared workflow-executor reusable workflow), including a new always-on step to log generation results.

Switches authentication from GITHUB_TOKEN/explicit workflow permissions to a GitHub App token (actions/create-github-app-token) so generated PRs can trigger downstream workflows (e.g., test suite), while keeping the same dispatch/scheduled triggers and kombo-python target.

Written by Cursor Bugbot for commit 07eeac4. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@maakle maakle requested a review from niklashigi February 19, 2026 17:08
@niklashigi niklashigi changed the title Change the workflow file for generating the SDK to directly handle it. chore: use custom GitHub App for update PRs so tests can run Feb 20, 2026
@niklashigi niklashigi merged commit 2658890 into main Feb 20, 2026
2 of 3 checks passed
@niklashigi niklashigi deleted the change-workflow branch February 20, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants